Marvell hwsku ET6448M 48x1G+4x10G Arm32 Mgmt switch#3221
Marvell hwsku ET6448M 48x1G+4x10G Arm32 Mgmt switch#3221lguohan merged 5 commits intosonic-net:masterfrom
Conversation
48x1G and 4x10G Management switch Signed-off-by: Antony Rheneus <arheneus@marvell.com>
Signed-off-by: Antony Rheneus <arheneus@marvell.com>
Signed-off-by: Antony Rheneus <arheneus@marvell.com>
| @@ -0,0 +1 @@ | |||
| switchMacAddress=00:50:43:ee:ee:ee | |||
There was a problem hiding this comment.
does it mean all switches will have the same mac?
There was a problem hiding this comment.
No, but Hardware doesn't have board eeprom, so its been updated in config file. Need to be updated per board.
There was a problem hiding this comment.
I do not quite understand the logic here, the mac address should be embedded in your board, we should not put a file in the image because then every image will be different for each board. but, I will let you address this problem later.
|
can you remove config_db.json and init_cfg.json, they are not needed in the image. |
Is it end user's responsibility to generate config_db.json? |
Signed-off-by: Antony Rheneus <arheneus@marvell.com>
Done, removed json files |
Signed-off-by: Antony Rheneus <arheneus@marvell.com>
|
can you update your pr title to be more specific? |
antony-rheneus
changed the title
Done |
antony-rheneus
changed the title
3 participants
- What I did
- How I did it
- How to verify it
- Description for the changelog
- A picture of a cute animal (not mandatory but encouraged)